Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Etherpadlite tinymce embed #26

Merged
merged 10 commits into from
Aug 29, 2013

Conversation

dtygel
Copy link

@dtygel dtygel commented Aug 26, 2013

Now Cirandas users will be able to insert directly a pad to any content created using TinyMCE. I'm using the plugin I've created and that is in https://github.com/dtygel/tinymce-etherpadlite-embed

@brauliobo
Copy link

dani, o plugin_etherpadlite_padServerUrl teria que vir de uma configuração no ambiente

@brauliobo
Copy link

o plugin_etherpadlite_padNamesPrefix tb

@brauliobo
Copy link

o ícone faria sentido usar o https://github.com/ether/etherpad-lite/blob/develop/src/static/favicon.ico para manter a coerencia com o etherpad-lite

@brauliobo
Copy link

dani, para atualizar este pull request é só fazer commits neste branch

@brauliobo
Copy link

dani, setei o campo e o botão para colocar um pad não apareceu

e=Environment.default
e.tinymce_plugin_etherpadlite_padServerUrl = 'pad.eita.org.br'
e.save

Deixo para você testar aí com o script/server

@brauliobo
Copy link

ah, outra coisa, o console javascript diz Uncaught SyntaxError: Unexpected token ( no editor_plugin.js

@dtygel
Copy link
Author

dtygel commented Aug 29, 2013

Oi Bráulio. É que eu tinha minificado o editor_plugin_src.js para editor_plugin.js e aí o .js sentiu a falta do ";". Eu percebi isso de cabeça, agora no ônibus. Chegando aqui na atividade em que estamos já fiz o pull, antes mesmo de ver tua mensagem. Não estou conseguindo testar com script/server, pois tá demorando séculos e estourando minha CPU fazer o 0.0.0.0:3000 no navegador. Me parece ter a ver com o fato de não ter imagens locais. demoooora e nada de funcionar....

Mas testei o plugin localmente e agora funcionou.

@brauliobo
Copy link

Ok Dani, vou testar.
Sobre as imagens coloque este código (mas não comite) na classe do app/models/image.rb

def public_filename *args
  "http://cirandas.net/#{super *args}"
end

brauliobo added a commit that referenced this pull request Aug 29, 2013
@brauliobo brauliobo merged commit f99f2c1 into CIRANDAS:master Aug 29, 2013
@brauliobo
Copy link

dani, preciso que vc crie um AI.

depois vou extrair os commits para fazer o merge request, pq se fizer do branch como um todo pega tb outros commits do cirandas... da próxima vez, o ideal é vc fazer a partir do noosfero limpo (http://gitorious.org/noosfero/noosfero), assim o pull request e o merge request saem do mesmo branch.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants